Skip to content

Comments

feat(AutoSign): Add AutoSign Module#236

Open
IceTank wants to merge 2 commits intolambda-client:1.21.11from
IceTank:feature/module/AutoSign
Open

feat(AutoSign): Add AutoSign Module#236
IceTank wants to merge 2 commits intolambda-client:1.21.11from
IceTank:feature/module/AutoSign

Conversation

@IceTank
Copy link
Contributor

@IceTank IceTank commented Feb 1, 2026

Add AutoSign Module to autofill sign texts and auto close ópened signs

@IceTank IceTank marked this pull request as ready for review February 1, 2026 19:18
@beanbag44 beanbag44 self-requested a review February 22, 2026 11:28
Copy link
Member

@beanbag44 beanbag44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, couple changes though

transitive-accessible field net/minecraft/client/gui/screen/ingame/HandledScreen focusedSlot Lnet/minecraft/screen/slot/Slot;
transitive-accessible field net/minecraft/registry/SimpleRegistry frozen Z

transitive-accessible field net/minecraft/client/gui/screen/ingame/AbstractSignEditScreen messages [Ljava/lang/String;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prob just remove the white space above to move it into the # Other group

else SignEditScreen(event.sign, true, mc.shouldFilterText())
for (i in 0 until 4) editor.messages[i] = lines[i]
if (autoClose) {
if (writeOnFront) mc.networkHandler?.sendPacket(UpdateSignC2SPacket(event.sign.pos, true, editor.messages[0], editor.messages[1], editor.messages[2], editor.messages[3]))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SafeContext contains connection which should be used instead of mc.networkHandler. Also 2b2t doesnt allow sign editing so this doesnt seem to work. Would it be possible to close the sign automatically without using edit sign?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants